Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swift 5.10 #74

Merged
merged 6 commits into from
Dec 4, 2024
Merged

Swift 5.10 #74

merged 6 commits into from
Dec 4, 2024

Conversation

rasberik
Copy link
Contributor

@rasberik rasberik commented Nov 28, 2024

Checklist

  • All tests are passed.
  • Added tests or Playbook scenario.
  • Documented the code using Xcode markup.
  • Searched existing pull requests for ensure not duplicated.

Why

Maintenance & deps update

Note

Im noticing some performance issues when building with Xcode 16.1.

Screenshots

@rasberik rasberik changed the title Swift.10 Swift 5.10 Nov 28, 2024
Copy link
Member

@ra1028 ra1028 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! @rasberik


jobs:
validation:
name: Validation
runs-on: macos-13
runs-on: macos-14
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rasberik
Can you update for Test on macOS job as well?

Copy link
Contributor Author

@rasberik rasberik Dec 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahhh Im making same mistakes everywhere. Thanks, updated

@ra1028
Copy link
Member

ra1028 commented Dec 4, 2024

@rasberik
It seems that an error happened with node package installation, so can you try updating percy version as well?

@rasberik
Copy link
Contributor Author

rasberik commented Dec 4, 2024

@ra1028 Do you think its ok to bump all the way 5 major versions?

-"percy": "0.28.4"  
+"percy": "5.0.0"

package.json Outdated
@@ -3,6 +3,6 @@
"version": "0.1.0",
"author": "Playbook",
"devDependencies": {
"percy": "0.28.4"
"percy": "5.0.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5 major version bump~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lockfile got shrinked massively tho~

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently, that because of this reason.
It seems that the package got renamed to @percy/cli.

Copy link
Contributor Author

@rasberik rasberik Dec 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I switched to @percy/cli-upload (pkg) to minimize deps, can we give it a try? Doesnt look like we need config update

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, that sounds good.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error still wants percy 5.0.. hmmmhmhmh what did I miss

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, cli is yappari requirement, fixed, please rerun 🙇🏻

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finally succeeded! There you go

@ra1028 ra1028 merged commit ef0398b into playbook-ui:master Dec 4, 2024
2 checks passed
@rasberik rasberik deleted the swift-5.10 branch December 5, 2024 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants